Skip to content

perf: increase Qwen3-235B async rollout GPU memory utilization on H100#3278

Open
humairafirdowse18 wants to merge 1 commit into
mainfrom
humairafirdo/perf-qwen3-235b-async-gpu-memory-util
Open

perf: increase Qwen3-235B async rollout GPU memory utilization on H100#3278
humairafirdowse18 wants to merge 1 commit into
mainfrom
humairafirdo/perf-qwen3-235b-async-gpu-memory-util

Conversation

@humairafirdowse18

Copy link
Copy Markdown

What does this PR do?

Increase policy.generation.vllm_cfg.gpu_memory_utilization from 0.80 to 0.88 for the H100 grpo-qwen3-235b-32n8g-async-1off performance recipe.

The additional vLLM memory allocation increases available KV-cache capacity and improves rollout generation throughput, reducing training buffer starvation.

Performance

Averaged over the measured steady-state steps:

Metric 0.80 0.88
E2E tokens/sec/GPU ~51–52 ~57.8
Exposed generation time ~174 s ~136–138 s

This is approximately an 11% improvement in E2E throughput.
W&B Logs: https://wandb.ai/nvidia/nemo-rl-perf/runs/lx9ha7b9?nw=nwuserhumairafirdo

Issues

N/A

Usage

No user-facing API changes. The optimization applies automatically when running:

examples/configs/recipes/llm/performance/grpo-qwen3-235b-32n8g-async-1off.yaml

Before your PR is "Ready for review"

  • Read and followed the contributor guidelines.
  • No new unit tests are required for this recipe-only configuration change.
  • Validated the change using the H100 performance run.
  • No documentation changes are required.

Additional Information

This PR contains a single performance-recipe configuration change.

Signed-off-by: humairafirdowse18 <humairafirdo@nvidia.com>
@humairafirdowse18
humairafirdowse18 requested a review from a team as a code owner July 18, 2026 02:23
@copy-pr-bot

copy-pr-bot Bot commented Jul 18, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@humairafirdowse18 humairafirdowse18 added CI:L1 Run doctests, unit tests, and functional tests CI:L2 Run doctests, unit tests, functional tests, and convergence tests and removed CI:L1 Run doctests, unit tests, and functional tests labels Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:L2 Run doctests, unit tests, functional tests, and convergence tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant